Skip to content

Conversation

WOLFRIEND
Copy link
Contributor

Описание

Добавление информации о таких возможностях как WeakRef & FinalizationRegistry.
Обсуждения и утверждение данного предложения находятся по ссылке:
#1898

Ссылки

Связанные Issue

@javascript-translate-bot javascript-translate-bot added the review needed Review needed, please approve or request changes label Oct 12, 2023
@javascript-translate-bot javascript-translate-bot requested a review from a team October 12, 2023 10:16
@CLAassistant
Copy link

CLAassistant commented Oct 12, 2023

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@Alexandre887 Alexandre887 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Добрый вечер.

Не пугайтесь большого количества правок — в основном это suggested changes, которые можно принять в один клик.

Также:

  • Я оставил комментарии к некоторым строкам кода, т.к не везде понятна ваша мысль.
  • Я оставил комментарии к некоторым частям текста, где есть тавтология и/или логические ошибки. Также где-то определённые выражения звучат не совсем по-русски, так что их нужно перефразировать.
  • Были ошибки, которые распространяются на весь материал статьи. К примеру, неверное написание определённых слов. В таком случае я не искал и не исправлял эти ошибки во всём тексте, просто оставил комментарий к одному подобному примеру, дальше вы должны будете сами их исправить.

Обязательно сначала примите suggested changes, а только потом рассматривайте комментарии. Иногда я оставлял и suggested changes, и комментарий в одной правке, так что внимательно всё прочтите.

Если мои замечания где-то не совсем понятны, или вы хотите их оспорить, — в Github также можно оставлять к ним комментарии, я получу уведомление и мы обсудим эти вопросы.

В техническую часть материала я пока глубоко не вдавался, сфокусировался на тексте и коде.

В раздел «Типы данных», на мой взгляд, статью будет помещать неправильно. Это очень узконаправленная тема, которая на практике не затрагивает 90% всех программистов. Многие спокойно пишут код, даже не зная, что такие возможности были введены в язык. Оно и к лучшему, т.к даже в mdn приведён перечень причин, по которым лучше избегать использования структур, о которых идет речь в статье.
Поэтому, лучше всего будет добавить статью в раздел «Разное», в конец первой части учебника.

В начале статьи можем добавить предостережение, указать в нём причины, которые я привёл выше. Также следует добавить информацию о том, что для прочтения необходимы базовые знания в DOM, т.к в статье есть примеры с его использованием.
Но, сейчас это не столь важно. Сосредоточьтесь на рассмотрении моих правок.

@javascript-translate-bot javascript-translate-bot added changes requested Waiting for changes and /done from PR author and removed review needed Review needed, please approve or request changes labels Oct 22, 2023
@javascript-translate-bot
Copy link
Contributor

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@WOLFRIEND
Copy link
Contributor Author

Спасибо большое за правки и предложение.
Я ознакомился и принял ваши suggested changes.
Теперь начинаю работу над вашими комментариями.

@WOLFRIEND
Copy link
Contributor Author

/done

@javascript-translate-bot javascript-translate-bot added review needed Review needed, please approve or request changes and removed changes requested Waiting for changes and /done from PR author labels Oct 25, 2023
@javascript-translate-bot javascript-translate-bot requested a review from a team October 25, 2023 08:30
@WOLFRIEND
Copy link
Contributor Author

Добрый день,

Я внёс изменения в соответствии с вашими правками:

  1. Принял ваши рекомендации.
  2. Исправил перевод текста в необходимых местах (картинки, примеры).
  3. Устранил ошибки, исправил непонятные описания и объяснения в соответствии с вашими комментариями.
  4. Перенёс статью в раздел, который вы предложили.
  5. Добавил необходимые предупреждения.

Я старался максимально следовать вашим правкам и комментариям. Пожалуйста, оцените результат и скажите, нужны ли дополнительные правки к материалу? Спасибо за вашу помощь!

@javascript-translate-bot javascript-translate-bot added changes requested Waiting for changes and /done from PR author and removed review needed Review needed, please approve or request changes labels Oct 29, 2023
@javascript-translate-bot
Copy link
Contributor

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@WOLFRIEND
Copy link
Contributor Author

/done

@javascript-translate-bot javascript-translate-bot added review needed Review needed, please approve or request changes and removed changes requested Waiting for changes and /done from PR author labels Oct 30, 2023
@javascript-translate-bot javascript-translate-bot requested a review from a team October 30, 2023 13:55
@WOLFRIEND
Copy link
Contributor Author

Добрый день,

Я внёс изменения в соответствии с вашими правками и ответил в комментариях.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs +1 One more review needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants